Try to increase quality of the codebase #1654
Conversation
* Now the code makes it more clear that this is backward compatibility support for CB1.
* Add specs to docblock * Remove unused imports
* This will be deprecated in future php versions (8.2) and may be removed later on. Maybe CB1 legacy code will stay a bit longer, so this could break.
* adds return statement
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1654 +/- ##
============================================
- Coverage 50.41% 50.39% -0.02%
+ Complexity 2719 2718 -1
============================================
Files 99 99
Lines 11296 11298 +2
============================================
- Hits 5695 5694 -1
- Misses 5601 5604 +3 ☔ View full report in Codecov by Sentry. |
datengraben
added
technical
…uctor also adds phpdoc
|
|
||
| // This was missing in previous versions. According to the availability spec, we can return a list with no items | ||
| // TODO this part and the enclosing if-clause can be removed in future version, if no problems arose ... | ||
| return new WP_REST_Response( $data, 200 ); |
There was a problem hiding this comment.
Funktioniert das auch? Also gibt der dann erfolgreich eine leere Liste zurück? Wenn ja kann das so bleiben.
| @@ -9,6 +9,8 @@ | |||
| */ | |||
| abstract class BaseShortcode { | |||
|
|
|||
| final public function __construct() { } | |||
There was a problem hiding this comment.
Der Konstruktor wird aber nicht nochmal von den erbenden Klassen benutzt, muss der dann trotzdem hier angegeben werden? Bzw welchen Sinn hat es dann ihn da anzugeben?
| @@ -286,7 +286,7 @@ public static function init( bool $ignoreErrors = false ):array{ | |||
| } | |||
| try { | |||
| $bookingRule->setAppliedParams( | |||
| $ruleParams ?? [], | |||
| $ruleParams, | |||
There was a problem hiding this comment.
Geht das dann immer noch mit Regeln die keine ruleParams definiert haben? Ich meine ich hab das eingeführt, weil es sonst einen fatal Error mit solchen Regeln gab.
| @@ -51,8 +52,7 @@ public static function getView() { | |||
| **/ | |||
| public static function replace_map_link_target() { | |||
There was a problem hiding this comment.
Die Funktion ist sowieso unbenutzt, nicht? Also lohnt es sich vielleicht gar nicht da was dran zu ändern.
| @@ -114,6 +119,8 @@ public function form( $instance ) { | |||
| </p> | |||
| <?php | |||
|
|
|||
| return ''; // Parent class returns string, not used | |||
There was a problem hiding this comment.
Für das UserWidget haben wir keinen automatisierten Test, geht das noch? Wenn ja passt das!
@hansmorb